Skip to content

test(integration): check for empty file id.#9738

Open
camilasan wants to merge 2 commits intomasterfrom
bugfix/noid/emptyfileid
Open

test(integration): check for empty file id.#9738
camilasan wants to merge 2 commits intomasterfrom
bugfix/noid/emptyfileid

Conversation

@camilasan
Copy link
Copy Markdown
Member

@camilasan camilasan commented Mar 30, 2026

Extends tests with checks for empty file id in standard sync and file provider.

Continuation of #9701

@camilasan camilasan added this to the 33.0.1 milestone Mar 30, 2026
@camilasan
Copy link
Copy Markdown
Member Author

/backport to stable-33.0

@nilsding nilsding modified the milestones: 33.0.1, 33.0.2, 33.0.3 Mar 31, 2026
@camilasan camilasan force-pushed the bugfix/noid/emptyfileid branch 2 times, most recently from 5fd2616 to e6e5383 Compare April 13, 2026 08:54
@camilasan camilasan modified the milestones: 33.0.3, 33.0.4 Apr 23, 2026
@mgallien mgallien force-pushed the bugfix/noid/emptyfileid branch from e6e5383 to 4b62860 Compare April 23, 2026 07:26
Signed-off-by: Camila Ayres <hello@camilasan.com>
The issue was only on macOS file provider.

Signed-off-by: Camila Ayres <hello@camilasan.com>
@nilsding nilsding force-pushed the bugfix/noid/emptyfileid branch from 4b62860 to 23bdf78 Compare April 27, 2026 11:08
@github-actions
Copy link
Copy Markdown

Artifact containing the AppImage: nextcloud-appimage-pr-9738.zip

Digest: sha256:b6e538c0c2033a8c6299ab8a460d7eaca65236f6754927948cc997d779f97a17

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)
12 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants